Feature/chia theme variants and overview - #3028
Conversation
Introduce an Overview landing page for full-node mode with live wallet, node, farm, and plot status while preserving all existing routes. Fix useStandardWallet to query the actual standard wallet id, and allow NFT fullscreen preview for non-image assets with contain fit in the dialog. Co-authored-by: Cursor <cursoragent@cursor.com>
Port the visual layer from the community GUI mod (akita-nft-preview-dialog branch): amber/green palette, dashboard chrome, sidebar, cards, tables, charts, and asset tinting. Add a browser design sandbox (npm run design:sandbox) for layout checks without a daemon. De-brand sample copy; keep the existing Overview and functional changes on this branch. Co-authored-by: Cursor <cursoragent@cursor.com>
Introduce a shared theme registry with per-variant palettes and typography so the GUI can switch between Classic, Autumn (field), and Chia branding at runtime. Co-authored-by: Cursor <cursoragent@cursor.com>
Expose Classic, Autumn, and Chia as selectable color themes in Settings, defaulting to Chia. Co-authored-by: Cursor <cursoragent@cursor.com>
Update sidebar, cards, buttons, and charts to use variant-aware colors and readable selection states across Classic, Autumn, and Chia themes. Co-authored-by: Cursor <cursoragent@cursor.com>
Load variant-specific logos and illustrations via ThemeAssetsContext so Classic, Autumn, and Chia each render matching artwork. Co-authored-by: Cursor <cursoragent@cursor.com>
Trim hero copy, fix overview card icon sizing, and add a hollow grid icon aligned with the sidebar icon set. Co-authored-by: Cursor <cursoragent@cursor.com>
Style token list selection and manage-tokens controls for readable contrast under the Chia palette. Co-authored-by: Cursor <cursoragent@cursor.com>
Align chart series and preview accents with the active theme variant primary palette.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
…riants-and-overview
cfbdbbf to
f744f24
Compare
| return paletteColor(theme, 'sidebarSelectedFill'); | ||
| } | ||
| return paletteColor(theme, 'sidebarBackground'); | ||
| } |
There was a problem hiding this comment.
Selected sidebar fill blends away
Medium Severity
selectedFill falls back to sidebarBackground when sidebarSelectedFill is missing, while the drawer paper is now also painted with sidebarBackground. For classic and field variants that omit sidebarSelectedFill, the selected item fill matches the drawer and no longer reads as selected (only the border remains).
Additional Locations (1)
Reviewed by Cursor Bugbot for commit f744f24. Configure here.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit f744f24. Configure here.
| }, | ||
| ); | ||
| /** @deprecated Import via theme registry. Kept for compatibility with older imports. */ | ||
| export { default } from './variants/field/dark'; |
There was a problem hiding this comment.
Wrong deprecated theme re-exports
Medium Severity
The deprecated light/dark entry points claim compatibility with older imports, but they now re-export the Autumn field variant. The previous modules were the original green palette, which this PR moved to classic. Callers that still import light or dark from @chia-network/core get amber field styling instead of the prior appearance.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit f744f24. Configure here.
3 participants
Note
Medium Risk
Large cross-cutting UI/theming change with a new default look and persisted preference handling; variant IDs are whitelisted, limiting injection risk, but visual regressions across themes/modes are the main concern.
Overview
Introduces a multi-variant theming system in
@chia-network/corewith whitelisted options Chia (new default), Classic, and Autumn (field), resolved through a central registry and applied from the GUI viaresolveAppThemetogether with light/dark mode.Users can switch variants in Settings → Color Theme (
ThemeVariantToggle); the choice is stored in local app prefs and validated withparseThemeVariantIdso unknown values cannot drive dynamic loading.Chia variant adds 2025 enterprise palette tokens, Poppins/Inter typography, variant-specific MUI themes, and GUI-bundled wordmark/icon assets wired through
ThemeAssetsProviderandThemedChiacomponents (replacing static@chia-network/iconslogos in key screens).Shared UI is updated to follow the active theme: sidebar/drawer styling, selected states on buttons and list items, line chart colors from
primary, and state/semantic colors from the palette rather than hard-coded greens.Legacy
light/dark/defaulttheme entry points remain as deprecated re-exports of the Autumn (field) variant. Adesign:sandboxscript and optional WalletConnect command-registry parity test (disabled) are included in the GUI package.Reviewed by Cursor Bugbot for commit f744f24. Bugbot is set up for automated code reviews on this repo. Configure here.